-
Notifications
You must be signed in to change notification settings - Fork 30k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
doc: add policy for landing new npm releases #21594
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The linter is failing, but pending that LGTM
doc/guides/maintaining-npm.md
Outdated
@@ -1,5 +1,11 @@ | |||
# Maintaining npm in Node.js | |||
|
|||
New pull requests should be opened when a "next" release has been released. | |||
|
|||
Once the "next" release has been promoted to "latest" the PR should be updated as neccessary |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nits:
- neccessary -> necessary
- Missing period.
- Wrapping this and the next line at 80 chars.
I'm gonna hold off on discussing this until Rebecca's back from PTO next week, but I'm enthusiastic about node having a more official policy about how we handle npm updates. |
I think this should be made clear in the document as well. |
This seems fine. I'm not sure if you'll want to have a policy on hotfixes/regressions, but otherwise the timing's pretty much up to y'all. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would like to have some text on how npm releases are backported, and a couple of notes on when/how to land a semver-major change in NPM.
This change in policy sets clear terms for when / how npm releases can be landed into master and how long they are expected to bake in the ecosystem. This is to cover all release types of npm including semver-major releases. What Node.js releases the updates land into are at the discretion of the release team.
Updated all the things. PTAL |
As @zkat says, npm is happy to go along with whatever seems best for the Node.js project. With my Node.js contributor hat on, this seems quite reasonable and feels to me like a comfortable margin to assess the safety of updates. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@nodejs/collaborators @nodejs/tsc @nodejs/npm I plan to land this in 24 hours if there are no objections |
assuming no major regressions are found. There are no additional constraints | ||
for Semver-Major releases. | ||
|
||
The specific Node.js release stream the new version will be able to land into |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
grammar nit: release streams … are?
Landed in 7570ad5 fixing above nit |
This change in policy sets clear terms for when / how npm releases can be landed into master and how long they are expected to bake in the ecosystem. This is to cover all release types of npm including semver-major releases. What Node.js releases the updates land into are at the discretion of the release team. PR-URL: #21594 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
This change in policy sets clear terms for when / how npm releases can be landed into master and how long they are expected to bake in the ecosystem. This is to cover all release types of npm including semver-major releases. What Node.js releases the updates land into are at the discretion of the release team. PR-URL: #21594 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Michael Dawson <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Franziska Hinkelmann <[email protected]> Reviewed-By: Tiancheng "Timothy" Gu <[email protected]>
This change in policy sets clear terms for when / how npm releases
can be landed into master and how long they are expected to bake in the
ecosystem. This is to cover all release types of npm including semver-major
releases.
What Node.js release streams the new version of npm land into are at the discretion of the
release team and not specified by this policy.
I would like to ensure that we have consensus in landing this from @nodejs/collaborators @nodejs/tsc @nodejs/release and @nodejs/npm
/cc @zkat @iarna @bcoe @ceejbot